-
-
Couldn't load subscription status.
- Fork 6.2k
Support actions and reusable workflows from private repos #32562
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
For changes in |
|
Is this related to #24635 ? |
|
Any progress for it? |
|
Any progress there? |
|
Any movement here? This would really really help us in our Gitea Cloud account. |
|
01211b3 to
82d368c
Compare
203ad9c to
01035bd
Compare
Could it be a private org is broken? Or is this nightly act_runner from https://dl.gitea.com/ not up to date.
|
I can't reproduce this issue. Can you make test01/test-repo private and try again? |
|
well good point, missed my own repo visibility. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, we could later enhance troubleshooting
- by log more information around repository not found
- if act_runner did an authenticated pull
|
act could check own repo visibility in it's payload when getting repository not found with token and log an actionable message |
| <div class="inline field"> | ||
| <label>{{ctx.Locale.Tr "actions.actions"}}</label> | ||
| <div class="ui checkbox{{if $isActionsGlobalDisabled}} disabled{{end}}"{{if $isActionsGlobalDisabled}} data-tooltip-content="{{ctx.Locale.Tr "repo.unit_disabled"}}"{{end}}> | ||
| <input class="enable-system" name="enable_actions" type="checkbox" {{if $isActionsGlobalDisabled}}disabled{{end}} {{if $isActionsEnabled}}checked{{end}}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is enable-system?
| {{if and .EnableActions (.Permission.CanRead ctx.Consts.RepoUnitTypeActions)}} | ||
| {{if .Repository.IsPrivate}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No indention?
| <div class="inline field"> | ||
| <label>{{ctx.Locale.Tr "actions.actions"}}</label> | ||
| <div class="ui checkbox{{if $isActionsGlobalDisabled}} disabled{{end}}"{{if $isActionsGlobalDisabled}} data-tooltip-content="{{ctx.Locale.Tr "repo.unit_disabled"}}"{{end}}> | ||
| <input class="enable-system" name="enable_actions" type="checkbox" {{if $isActionsEnabled}}checked{{end}}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't it cause bug since you remove this block? RepoSettingForm.EnableActions is always false now, then any edit on this page will disable the Actiosn unit?
RepoSettingForm.EnableActions is left there and need to be removed.
| return perm, err | ||
| } | ||
| actionsCfg := repo.MustGetUnit(ctx, unit.TypeActions).ActionsConfig() | ||
| if !actionsCfg.IsCollaborativeOwner(taskRepo.OwnerID) || !taskRepo.IsPrivate { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it right if the owner's visibility is not public?
|
I "fixed" these reviews in Remove unnecessary code #35761 |


Resolve https://gitea.com/gitea/act_runner/issues/102
This PR allows administrators of a private repository to specify some collaborative owners. The repositories of collaborative owners will be allowed to access this repository's actions and workflows.
Settings for private repos:
This PR also moves "Enable Actions" setting to
Actions > Generalpage